ci: bump setup-soldr v0.9.24 → v0.9.35 (~7s warm-run win)#333
Conversation
Picks up the cumulative perf + observability wins: - #302/#303: sub-phase timing observability (each setup phase now prints a `{sub=Xs sub=Ys}` breakdown inline, surfacing the slow spots). - #304/#308: FS-first rustup probe — eliminates ~7s `rustup toolchain list` subprocess on every warm CI run by reading $RUSTUP_HOME/toolchains/ directly. Validated on zccache CI: linux/Test wall clock dropped 30.0s → 14.8s (-15.2s). Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (11)
📝 WalkthroughWalkthroughAll CI and build workflows are updated to use ChangesSoldr setup action version bump across CI workflows
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Bumps
zackees/setup-soldrfromv0.9.24tov0.9.35across all.github/workflows/pins.What's in v0.9.35 (vs v0.9.24)
#304/#308 — FS-first rustup probe (the big win)
Eliminates the ~7s
rustup toolchain listsubprocess that fired on every warm CI run by reading$RUSTUP_HOME/toolchains/directly. Validated on zccache CI: linux/Test wall clock dropped 30.0s → 14.8s (-15.2s).#302/#303 — Sub-phase timing observability
Each setup phase now prints a
{sub=Xs sub=Ys}breakdown inline, so the slow spots in resolve/toolchain phases are visible without enabling debug mode.Scope
Mechanical bump only — replaces
zackees/setup-soldr@v0.9.24→zackees/setup-soldr@v0.9.35in 11 workflow files. No other changes.Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit